Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: fix lint warnings #1112

Merged
merged 16 commits into from
Dec 18, 2023
Merged

fix: fix lint warnings #1112

merged 16 commits into from
Dec 18, 2023

Conversation

Fupete
Copy link
Collaborator

@Fupete Fupete commented Dec 1, 2023

see #1108 (this PR fix half of those linting issues).
see this comment #1108 (comment)

@Fupete Fupete requested a review from bfabio December 1, 2023 09:03
@Fupete Fupete linked an issue Dec 1, 2023 that may be closed by this pull request
@Fupete

This comment was marked as outdated.

Copy link
Contributor

github-actions bot commented Dec 1, 2023

⚡ Lighthouse report for the home page 🏠

Category Score
🟠 Performance 54
🟢 Accessibility 100
🟢 Best practices 100
🟢 SEO 100

See the full report...

Other pages

@Fupete Fupete changed the title fix: remove all the linting warnings fix: fix all linting warnings Dec 7, 2023
@Fupete Fupete changed the title fix: fix all linting warnings fix: fix linting warnings Dec 18, 2023
@Fupete Fupete requested a review from astagi December 18, 2023 12:00
@Fupete Fupete changed the title fix: fix linting warnings fix: fix lint warnings Dec 18, 2023
@Fupete Fupete self-assigned this Dec 18, 2023
@Fupete Fupete enabled auto-merge (squash) December 18, 2023 14:46
@Fupete Fupete merged commit c860934 into main Dec 18, 2023
4 checks passed
@Fupete Fupete deleted the fix--remove-all-the-linting-warnings branch December 18, 2023 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants